Introduce Asian directory server in code#3697
Conversation
For geo redundancy, latency and since Any Genre 3 is not heavily used, we introduce a directory in Asia. See jamulussoftware#3692
|
Any Genre Asia shows up correctly in the client and server instead of Any Genre 3 |
| case AT_ANY_GENRE2: | ||
| return CENTSERV_ANY_GENRE2; | ||
| case AT_ANY_GENRE3: | ||
| return CENTSERV_ANY_GENRE3; |
There was a problem hiding this comment.
We should probably rather not name it CENTSERV anymore...
There was a problem hiding this comment.
Yeah, I was going to say - separate PR to move to DIRECTORY_ and .app in one go?
There was a problem hiding this comment.
We could do this yes. But .app didn't resolve for me yet. @gilgongo
There was a problem hiding this comment.
Ah, looking at the Cloudflare config- I'd forgotten that it works using 301 redirects. For some reason I thought it had some DNS wildcard capability. So no .app for directories right now I'm afraid.
There was a problem hiding this comment.
Yeah, I was going to say - separate PR to move to
DIRECTORY_and.appin one go?
I suppose we could replicate the .io zone to .app and maintain both in parallel? We don't make changes that often after all. As far as I can tell, CloudFlare don't allow pointing two domains to the same zone file at least.
There was a problem hiding this comment.
I'd like to get rid of mentioning anygenre3 anywhere on jamulus.app but not on jamulus.io.
anygenre3.jamulus.app never existed. One could also argue that asia.jamulus.io never existed (in a released version)
There was a problem hiding this comment.
Ah, that's true. So leave the SRV on jamulus.io, with the CNAME there both pointing to asia.jamulus.app and no other mention of asia.jamulus.app - and no mention of asia.jamulus.io.
There was a problem hiding this comment.
OK I can delete SRV _jamulus._udp.asia on jamulus.io and the CNAME for asia.jamulus.io? No anygenre3 records on .app.
There was a problem hiding this comment.
OK I can delete SRV _jamulus._udp.asia on jamulus.io and the CNAME for asia.jamulus.io? No anygenre3 records on .app.
Oh, that took me by surprise rather:
Changes like that shouldn't happen quite so fast - I don't spend all my time at the computer, despite occasionally appearing to do so!
There was a problem hiding this comment.
Sorry about that. I've not removed the SRV yet though.
softins
left a comment
softins
left a comment
There was a problem hiding this comment.
Looks fine and runs as expected, for both Client and Server modes.
4 participants
Short description of changes
For geo-redundancy, latency and since Any Genre 3 is not heavily used, we introduce a directory in Asia. See
#3692
CHANGELOG: Introduce Directory Server in Asia. This replaces Any Genre 3. Asian/Australian users should from now on use Any Genre Asia.
Context: Fixes an issue?
Does this change need documentation? What needs to be documented and how?
Fixes: #3692
Status of this Pull Request
Ready for review
What is missing until this pull request can be merged?
Review
Checklist